Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

enocean: modify connect logics, include refactoring by hasenradball #944

Merged
merged 5 commits into from
Jun 30, 2024
Merged

enocean: modify connect logics, include refactoring by hasenradball #944

merged 5 commits into from
Jun 30, 2024

Conversation

Morg42
Copy link
Member

@Morg42 Morg42 commented Jun 27, 2024

Should be able to handle disconnects/removal of EnOcean device and recovery if found again. Configurable retry and retry_cycle parameters.

Modified main run() loop to try reconnecting,

Refactored, fixed and cleaned up

included refactoring by hasenradball (move CRC and constants to protocol submodule)

@hasenradball
Copy link
Contributor

@Morg42,
ah ich sehe gerade Du hast meine Sachen übernommen ( Kommentar).
Danke, bin gerade etwas außer gefecht gesetzt mit nur einem Arm programmiert es sich nicht so gut..

@hasenradball
Copy link
Contributor

@Morg42
Hi, kurze Frage, warum hast du die CRC Table nicht in die Constants gepackt?
Ist doch auch eine Art Konstante, oder?

@Morg42
Copy link
Member Author

Morg42 commented Jun 29, 2024

Weil das Werte sind, die ausschließlich lokal verwendet werden und auch keine ggf zu ändernden Parameter sind. Würde constants.py nur unübersichtlicher machen

@hasenradball
Copy link
Contributor

Weil das Werte sind, die ausschließlich lokal verwendet werden und auch keine ggf zu ändernden Parameter sind. Würde constants.py nur unübersichtlicher machen

Ok passt, hab auch gesehen Du hast eine call methode eingebaut.
👍

@Morg42
Copy link
Member Author

Morg42 commented Jun 30, 2024

@hasenradball Hast du mal testen können, ob das bei dir alles funktioniert? Nicht, dass irgendwas "nebenher" kaputt gegangen ist? Ich habe nur Fenstergriffe, kann insofern keine sonstigen Geräte testen...

@hasenradball
Copy link
Contributor

@Morg42
Hi, hi konnte leider noch nicht testen.
wie heißt der branch enocean, richtig?

@hasenradball
Copy link
Contributor

hasenradball commented Jun 30, 2024 •
edited
Loading

@Morg42 helf mir mal bitte kurz auf die Sprünge, kann ich dein branch bei mir direkt auschecken?
Oder muss ich dazu dein Repo in plugins clonen?

@Morg42
Copy link
Member Author

Morg42 commented Jun 30, 2024

Auschecken? Nein, nur, wenn du erst mein Repo gecloned hast:

git checkout -b enocean https://github.com/Morg42/plugins.git

Danach das Plugin aus meinem Repo in dein Arbeitsverzeichnis kopieren...

Ansonsten gab es dazu im Forum mal eine Anleitung...

@bmxp
Copy link
Member

bmxp commented Jun 30, 2024

Ansonsten gab es dazu im Forum mal eine Anleitung...

Siehe https://knx-user-forum.de/forum/supportforen/smarthome-py/1695551-privates-plugin-zum-testen-ins-eigene-repo-herunterladen

@hasenradball
Copy link
Contributor

hasenradball commented Jun 30, 2024 •
edited
Loading

@Morg42
@bmxp

Danke habs mittels eines zweiten remote hinbekommen.

origin  https://github.com/smarthomeNG/plugins.git (fetch)
origin  https://github.com/smarthomeNG/plugins.git (push)
test    https://github.com/Morg42/plugins.git (fetch)
test    https://github.com/Morg42/plugins.git (push)

(py_shng) smarthome@pi-smarthome2:/usr/local/smarthome/plugins $ git status
HEAD losgelöst bei test/enocean
nichts zu committen, Arbeitsverzeichnis unverändert
(py_shng) smarthome@pi-smarthome2:/usr/local/smarthome/plugins $

Also ich habe den enocean branch gerade reingezogen beim reboot hatte ich nur beim AVM plugin einige errors, aber enochean lief ohne Auffälligkeiten.

Ich würde vorschlagen den enocen branch mal in den develop zu mergen und ich lass das enocean mal einige Wochen im produktiv System laufen.

Sieht gut aus.

@Morg42 Morg42 merged commit eee360d into smarthomeNG:develop Jun 30, 2024
4 checks passed
@hasenradball
Copy link
Contributor

@Morg42 dann kann ich mein usprünglichen branch ja löschen, korrekt?

@hasenradball
Copy link
Contributor

@Morg42
@bmxp
Habe nun das Produktivsystem auf develop umgestellt.
Keine Fehler von enocean.

Sagt euch das etwas:

04:06 NOTICE   lib.smarthome       --------------------   Init SmartHomeNG v1.10.0-master (4b25822a0)   --------------------
2024-07-02  18:04:06 NOTICE   lib.smarthome       Running in Python interpreter 'v3.11.2 final' in virtual environment, from directory /usr/local/smarthome
2024-07-02  18:04:06 NOTICE   lib.smarthome        - operating system 'Debian GNU/Linux 12 (bookworm)' (pid=167179)
2024-07-02  18:04:06 NOTICE   lib.smarthome        - on 'Raspberry Pi 3 Model B v1.2, 1GB (Sony, UK)'
2024-07-02  18:04:10 NOTICE   lib.smarthome        - Nutze Feiertage für Land 'DE', Provinz 'None', 1 benutzerdefinierte(r) Feiertag(e) definiert
2024-07-02  18:04:36 NOTICE   lib.smarthome       --------------------   SmartHomeNG initialization finished   --------------------
2024-07-02  18:04:36 WARNING  lib.item.item       Item 'EG.Wohnen.FritzDect.uzsu.next.value': problem evaluating 'sh.EG.Wohnen.FritzDect.uzsu.next.property.value['value']' - KeyError (in dict)
2024-07-02  18:04:36 WARNING  lib.item.item       Item 'EG.Wohnen.FritzDect.uzsu.next.time': problem evaluating 'sh.EG.Wohnen.FritzDect.uzsu.next.property.value['next']' - KeyError (in dict)
2024-07-02  18:04:36 WARNING  lib.item.item       Item 'OG.Flur.RollladenZentral.Lauf.uzsu.next.value': problem evaluating 'sh.OG.Flur.RollladenZentral.Lauf.uzsu.next.property.value['value']' - KeyError (in dict)
2024-07-02  18:04:36 WARNING  lib.item.item       Item 'OG.Flur.RollladenZentral.Lauf.uzsu.next.time': problem evaluating 'sh.OG.Flur.RollladenZentral.Lauf.uzsu.next.property.value['next']' - KeyError (in dict)
2024-07-02  18:04:36 WARNING  lib.item.item       Item 'UG.Technik.WW_Pumpe.uzsu.next.value': problem evaluating 'sh.UG.Technik.WW_Pumpe.uzsu.next.property.value['value']' - KeyError (in dict)
2024-07-02  18:04:36 WARNING  lib.item.item       Item 'UG.Technik.WW_Pumpe.uzsu.next.time': problem evaluating 'sh.UG.Technik.WW_Pumpe.uzsu.next.property.value['next']' - KeyError (in dict)

@smarthomeNG smarthomeNG locked and limited conversation to collaborators Jul 2, 2024
@Morg42 Morg42 deleted the enocean branch July 3, 2024 06:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Morg42 @hasenradball @bmxp